Skip to content

chore: Add deployer name in Createdby tag#433

Merged
Roopan-Microsoft merged 1 commit into
devfrom
psl-CreatedBy-Tag
Sep 3, 2025
Merged

chore: Add deployer name in Createdby tag#433
Roopan-Microsoft merged 1 commit into
devfrom
psl-CreatedBy-Tag

Conversation

@NirajC-Microsoft

@NirajC-Microsoft NirajC-Microsoft commented Sep 3, 2025

Copy link
Copy Markdown
Contributor

Purpose

This pull request introduces tagging enhancements to the infrastructure deployment process in infra/main.bicep. The main change is the addition of a resource to apply standardized and informative tags to the resource group, improving resource management and traceability.

Tagging improvements:

  • Added a new resourceGroupTags resource that applies a set of tags to the resource group, including environment name (azd-env-name), template name (TemplateName), and the user who created the deployment (CreatedBy, derived from the deployer's principal name).
  • Consolidated custom and inherited tags using the union function for comprehensive tagging.
  • ...

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

[Copilot is generating a summary...]

@Roopan-Microsoft Roopan-Microsoft merged commit 32989a9 into dev Sep 3, 2025
8 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the psl-CreatedBy-Tag branch September 3, 2025 06:19
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

deepammi pushed a commit to deepammi/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator that referenced this pull request Dec 7, 2025
chore: Add deployer name in Createdby tag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants